Skip to content

docs: fix data classification reference link - #2774

Merged
Jamie Kim (jkim323) merged 7 commits into
microsoft:mainfrom
icecold009:codex/hve-core-2758-link-fix
Sep 5, 2026
Merged

docs: fix data classification reference link#2774
Jamie Kim (jkim323) merged 7 commits into
microsoft:mainfrom
icecold009:codex/hve-core-2758-link-fix

Conversation

@icecold009

Copy link
Copy Markdown
Contributor

Summary

  • Replace the broken Microsoft Learn data-classification URL with the current taxonomy page.
  • Narrow the surrounding public-material wording to categories and labels documented by that source.
  • Refresh the reference date.

Closes #2758

Testing

  • npx --no-install markdownlint-cli2 .github/skills/project-planning/security-planning/references/data-classification.md — passed.
  • npx --no-install cspell .github/skills/project-planning/security-planning/references/data-classification.md — passed.
  • git diff --check — passed.
  • Targeted Markdown link checking was blocked locally because both Microsoft Learn requests returned status 0; the replacement URL was verified separately against the official Microsoft Learn page.

@icecold009
Shaurya Saria (icecold009) requested a review from a team as a code owner August 24, 2026 14:17

@jkim323 Jamie Kim (jkim323) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Shaurya Saria (@icecold009) for working on this PR. So far looks great but have a couple asks for you to review. Please ensure you also follow the PR Template guidelines and update with following the local checks required. Happy to review again after you take a look at the comments!

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.24%. Comparing base (bb0ac58) to head (1bfcc4e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2774      +/-   ##
==========================================
+ Coverage   83.14%   83.24%   +0.09%     
==========================================
  Files         183      172      -11     
  Lines       34039    34350     +311     
  Branches       27        0      -27     
==========================================
+ Hits        28302    28594     +292     
- Misses       5734     5756      +22     
+ Partials        3        0       -3     
Flag Coverage Δ
docusaurus ?
pester 84.24% <ø> (+0.40%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 20 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jkim323
Jamie Kim (jkim323) merged commit aaf51b9 into microsoft:main Sep 5, 2026
118 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: broken Microsoft Learn data classification link in security-planning skill reference

4 participants